Hi Masahiro,
> + #elif defined(__INTEL_COMPILER)
> + /* How to get the version of intel compiler? */
> + ICC 0 0 0
According to Intel documentation[1], this should do the trick:
ICC __INTEL_COMPILER __INTEL_COMPILER_UPDATE
The following commit has been merged into the objtool/core branch of tip:
Commit-ID: 14db1f0a93331d0958e90da522c429ff0890d2d6
Gitweb:
https://git.kernel.org/tip/14db1f0a93331d0958e90da522c429ff0890d2d6
Author:Ilie Halip
AuthorDate:Sat, 19 Sep 2020 09:41:18 +03:00
Desaulniers
Reported-by: kbuild test robot
Signed-off-by: Ilie Halip
---
Changed in v2:
- added a mention that this is a clang issue across all versions
- added Nick's Reviewed-by, Tested-by
- added Reported-by
tools/objtool/check.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions
> The patch looks good to me. Which versions of Clang do the trap after
> noreturn call? It would be good to have that in the commit message.
I omitted this because it happens with all versions of clang that are
supported for building the kernel. clang-9 is the oldest version that
could build
/CAKwvOdmptEpi8fiOyWUo=aizjix+z+vhjom2bulprwsmtwl...@mail.gmail.com
Suggested-by: Nick Desaulniers
Signed-off-by: Ilie Halip
---
tools/objtool/check.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/tools/objtool/check.c b/tools/objtool/check.c
index e034a8f24f46..eddf8bf16b05
> Should objtool be made aware of the config option and then not check
> traps after no-returns?
>
> I suspect the latter, but I'm not sure how feasible it is to
> implement. Josh, Marco, do you have thoughts on the above?
This seems to do the trick.
diff --git a/tools/objtool/check.c
Hi Nick,
> Ilie, may I put your authorship and signed off by tag on the V2?
Yes, of course. With the current global situation I took some time off
and didn't follow the latest discussion.
Feel free to credit/rework the code as you see fit.
Thanks,
I.H.
into a section which does the same macro check.
Reported-by: Nathan Chancellor
Signed-off-by: Ilie Halip
---
arch/powerpc/platforms/powermac/smp.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/powerpc/platforms/powermac/smp.c
b/arch/powerpc/platforms/powermac/smp.c
> > $ clang-9 -c crc32.i -O2 ; objtool check crc32.o
> > crc32.o: warning: objtool: fn1 uses BP as a scratch register
Yes, I see it too. https://godbolt.org/z/N56HW1
> Do you still see this warning with -fno-omit-frame-pointer (assuming
> clang has that option)?
Using this makes the
: Section mismatch in reference from the function weim_probe() to
the function .init.text:weim_timing_setup()
Remove the __init attribute from these functions as well, since they
don't seem to be used anywhere else.
Signed-off-by: Ilie Halip
Reported-by: "kernelci.org bot"
Cc: Sascha
10 matches
Mail list logo
