[clang] 246398e - [clang][Parse] properly parse asm-qualifiers, asm inline
Joerg Sonnenberger via cfe-commits
cfe-commits at lists.llvm.org
Sun Apr 12 15:45:02 PDT 2020
On Thu, Mar 12, 2020 at 03:25:49PM -0700, Nick Desaulniers via cfe-commits wrote:
>
> Author: Nick Desaulniers
> Date: 2020-03-12T15:13:59-07:00
> New Revision: 246398ece7115b57a02dbe7876d86ae8e72406ef
>
> URL: https://github.com/llvm/llvm-project/commit/246398ece7115b57a02dbe7876d86ae8e72406ef
> DIFF: https://github.com/llvm/llvm-project/commit/246398ece7115b57a02dbe7876d86ae8e72406ef.diff
>
> LOG: [clang][Parse] properly parse asm-qualifiers, asm inline
>
> Summary:
> The parsing of GNU C extended asm statements was a little brittle and
> had a few issues:
I find it very questionable that this change drops a warning flag making
IMO benign code differences like marking a global asm as volatile an
unconditional error. This strongly seems to be something that we
shouldn't blindly follow GCC on.
Joerg
More information about the cfe-commits
mailing list