diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2016-05-10 18:22:45 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2016-05-10 18:22:45 +0000 |
| commit | 32483a7641a431bf9e217948fb237642a26c7569 (patch) | |
| tree | 8425125c34e5dc4ca1394bb4b599727e88fc90a7 /llvm/test/Assembler/invalid-hexint.ll | |
| parent | c3b0a5b0999f0783ab574094bcbe70d20e37d44c (diff) | |
| download | bcm5719-llvm-32483a7641a431bf9e217948fb237642a26c7569.tar.gz bcm5719-llvm-32483a7641a431bf9e217948fb237642a26c7569.zip | |
Make "@name =" mandatory for globals in .ll files.
An oddity of the .ll syntax is that the "@var = " in
@var = global i32 42
is optional. Writing just
global i32 42
is equivalent to
@0 = global i32 42
This means that there is a pretty big First set at the top level. The
current implementation maintains it manually. I was trying to refactor
it, but then started wondering why keep it a all. I personally find the
above syntax confusing. It looks like something is missing.
This patch removes the feature and simplifies the parser.
llvm-svn: 269096
Diffstat (limited to 'llvm/test/Assembler/invalid-hexint.ll')
| -rw-r--r-- | llvm/test/Assembler/invalid-hexint.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/Assembler/invalid-hexint.ll b/llvm/test/Assembler/invalid-hexint.ll index a11b8cd0e88..f4d52ba967a 100644 --- a/llvm/test/Assembler/invalid-hexint.ll +++ b/llvm/test/Assembler/invalid-hexint.ll @@ -1,4 +1,4 @@ ; RUN: not llvm-as < %s 2>&1 | FileCheck %s -global i64 u0x0p001 +@0 = global i64 u0x0p001 ; CHECK: expected value token |
