Thread (26 messages) flat view 26 messages, 7 authors, 2018-09-20

Re: [PATCH v2 2/5] powerpc/boot: Fix crt0.S syntax for clang

From: Nick Desaulniers <ndesaulniers@google.com>
Date: 2018-09-17 20:41:48

On Fri, Sep 14, 2018 at 2:08 PM Segher Boessenkool
[off-list ref] wrote:
On Fri, Sep 14, 2018 at 10:47:08AM -0700, Nick Desaulniers wrote:
quoted
On Thu, Sep 13, 2018 at 9:07 PM Joel Stanley [off-list ref] wrote:
quoted
 10:    addis   r12,r12,(-RELACOUNT)@ha
-       cmpdi   r12,RELACOUNT@l
+       cmpdi   r12,(RELACOUNT)@l
Yep, as we can see above, when RELACOUNT is negated, it's wrapped in
parens.
Looks like this was just fixed in Clang-8:
https://bugs.llvm.org/show_bug.cgi?id=38945
https://reviews.llvm.org/D52188

-- 
Thanks,
~Nick Desaulniers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help